Skip to content

Enable -mergefunc-use-aliases#56358

Merged
bors merged 2 commits intorust-lang:masterfrom
nikic:mergefunc-aliases
Dec 3, 2018
Merged

Enable -mergefunc-use-aliases#56358
bors merged 2 commits intorust-lang:masterfrom
nikic:mergefunc-aliases

Conversation

@nikic
Copy link
Contributor

@nikic nikic commented Nov 29, 2018

If the Rust LLVM fork is used, enable the -mergefunc-use-aliases
flag, which will create aliases for merged functions, rather than
inserting a call from one to the other.

A number of codegen tests needed to be adjusted, because functions
that previously fell below the thunk limit are now being merged.
Merging is prevented in various ways now.

I expect that this is going to break something, somewhere, because
it isn't able to deal with aliases properly, but we won't find out
until we try :)

This fixes #52651.

r? @rkruppe

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

rustc beta (1.28.0) generates unnecessary call instruction

7 participants